Skip to content

🐛 Fix terraform panic when using terraform.block#5491

Merged
claytoncasey01 merged 3 commits intomainfrom
claytoncasey01/terraform-segfault
May 1, 2025
Merged

🐛 Fix terraform panic when using terraform.block#5491
claytoncasey01 merged 3 commits intomainfrom
claytoncasey01/terraform-segfault

Conversation

@claytoncasey01
Copy link
Copy Markdown
Contributor

@claytoncasey01 claytoncasey01 commented Apr 29, 2025

The autocomplete mistakenly showed terraform.block instead of terraform.blocks when running cnquery shell. Using terraform.block would cause a panic. This has been updated to not display terraform.block and if it is used show a helpful error.

Screenshot 2025-04-29 at 8 25 34 AM

@claytoncasey01 claytoncasey01 changed the title 🐛 Fix terraform 🐛 Fix terraform panic when using terraform.block Apr 29, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2025

Test Results

4 230 tests  ±0   4 226 ✅ ±0   2m 6s ⏱️ -2s
  402 suites ±0       4 💤 ±0 
   30 files   ±0       0 ❌ ±0 

Results for commit 56b322b. ± Comparison against base commit f0121a5.

♻️ This comment has been updated with latest results.

@vjeffrey vjeffrey requested a review from afiune April 29, 2025 19:24
Copy link
Copy Markdown
Contributor

@afiune afiune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@claytoncasey01 claytoncasey01 merged commit da77d51 into main May 1, 2025
17 checks passed
@claytoncasey01 claytoncasey01 deleted the claytoncasey01/terraform-segfault branch May 1, 2025 21:34
@github-actions github-actions Bot locked and limited conversation to collaborators May 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants